Skip to content

Make Ask Loci toggle work outside the workspace - #5

Merged
PermutationResearch merged 1 commit into
mainfrom
fix/ask-loci-toolbar-toggle
Jul 10, 2026
Merged

Make Ask Loci toggle work outside the workspace#5
PermutationResearch merged 1 commit into
mainfrom
fix/ask-loci-toolbar-toggle

Conversation

@PermutationResearch

Copy link
Copy Markdown
Owner

Summary

  • The Ask Loci toolbar button (and ⌥⌘I / command palette action) posted a notification whose only listener lives inside ChatWorkspaceView, which exists only while Ask Loci is already open — so the controls did nothing from the library
  • Add a shell-level listener that enters the Ask Loci workspace (mirroring the sidebar row, including resetting the inspector-visible flag); the existing in-workspace listener still toggles the chat panel

Testing

  • swift build -c release clean
  • swift test — 23/23 passing

🤖 Generated with Claude Code

The toolbar button, option-command-I menu item, and command palette
action post lociToggleNotebookInspector, whose only listener lives
inside ChatWorkspaceView — a view that exists only while Ask Loci is
already open, so the controls silently did nothing elsewhere. Listen
at the shell level and enter the workspace the way the sidebar row
does; inside Ask Loci the existing listener still toggles the chat
panel.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@PermutationResearch
PermutationResearch merged commit d41e6b2 into main Jul 10, 2026
1 check passed
@PermutationResearch
PermutationResearch deleted the fix/ask-loci-toolbar-toggle branch July 10, 2026 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants